A new Pull Request was created by @AdrianoDee for CMSSW_14_0_X.

It involves the following packages:

• Geometry/CommonTopologies (geometry)
• RecoLocalTracker/SiPixelClusterizer (reconstruction)
• RecoLocalTracker/SiPixelRecHits (reconstruction)
• RecoTracker/PixelSeeding (reconstruction)

@makortel, @jfernan2, @Dr15Jones, @civanch, @bsunanda, @cmsbuild, @mandrenguyen, @mdhildreth can you please review it and eventually sign? Thanks.
@JanFSchulte, @tvami, @rovere, @gpetruc, @ferencek, @GiacomoSguazzoni, @bsunanda, @felicepantaleo, @mtosi, @VinInn, @fabiocos, @dgulhan, @mmusich, @missirol, @VourMa, @dkotlins, @tsusa, @threus, @mroguljic this is something you requested to watch as well.
@antoniovilela, @sextonkennedy, @rappoccio you are the release manager for this.

cms-bot commands are listed here

• Backported from Allow Runtime Number of Hits for Alpaka Pixel Reconstruction #44773

cms-bot internal usage

tagging @malbouis (ORM)

enable gpu

backport of #44773

test parameters:

• enable = gpu
• workflows = 12434.402,12434.403,12434.404
• workflows_gpu = 12434.402,12434.403,12434.404
• workflow_opts = -w upgrade
• workflow_opts_gpu = -w upgrade

Pull request #44774 was updated. @bsunanda, @jfernan2, @civanch, @makortel, @Dr15Jones, @cmsbuild, @mdhildreth, @mandrenguyen can you please check and sign again.

urgent

code-checks

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-44774/40007

• This PR adds an extra 40KB to repository

please test

assign heterogeneous

New categories assigned: heterogeneous

@fwyzard,@makortel you have been requested to review this Pull request/Issue and eventually sign? Thanks

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-295fff/38936/summary.html
COMMIT: 72591c3
CMSSW: CMSSW_14_0_X_2024-04-18-1100/el8_amd64_gcc12
Additional Tests: GPU
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-sw/cmssw/44774/38936/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

Summary:

• You potentially removed 89 lines from the logs
• Reco comparison results: 48 differences found in the comparisons
• DQMHistoTests: Total files compared: 50
• DQMHistoTests: Total histograms compared: 3350877
• DQMHistoTests: Total failures: 18
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 3350839
• DQMHistoTests: Total skipped: 20
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.0 KiB( 49 files compared)
• Checked 213 log files, 177 edm output root files, 50 DQM output files
• TriggerResults: no differences found

GPU Comparison Summary

Summary:

• No significant changes to the logs found
• Reco comparison results: 58 differences found in the comparisons
• DQMHistoTests: Total files compared: 5
• DQMHistoTests: Total histograms compared: 71813
• DQMHistoTests: Total failures: 2964
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 68849
• DQMHistoTests: Total skipped: 0
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.0 KiB( 4 files compared)
• Checked 19 log files, 22 edm output root files, 5 DQM output files
• TriggerResults: no differences found

The master PR is merged. If there is time before building the patch release, check the master IB.
The patch release should add only this PR (#44774) and #44768, as requested by ORM.

lets get this merged. I will also include it on top of CMSSW_14_0_5_patchX branch ( which is based on CMSSW_14_0_5_patch1 and will only only #44774 and #44768) . in few minutes next IB for 14.0.x and 14.1.X will be build and if every thing goes fine there then I will start the 14.0.5.patch2 build

lets get this merged. I will also include it on top of CMSSW_14_0_5_patchX branch ( which is based on CMSSW_14_0_5_patch1 and will only only #44774 and #44768) . in few minutes next IB for 14.0.x and 14.1.X will be build and if every thing goes fine there then I will start the 14.0.5.patch2 build

Ok, perfect.

@antoniovilela thank you for being proactive on this.

As far as I know we have not yet reviewed the changes and the overall impact on the HLT.

+1

+heterogeneous